-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
adding traces in SpMorphicMillerAdapter to trace problems in CI #665
adding traces in SpMorphicMillerAdapter to trace problems in CI #665
Conversation
do you need it in Pharo or just your PR is enough to get the trace? |
This is just for me to investigate so I think the PR should be enough (and if it should be in Pharo, I should do a PR in pharo-spec/Spec instead ). In |
Example of CI run that went wrong with traces: https://github.com/pharo-spec/NewTools/actions/runs/7611736592/job/20727806873?pr=665 Example of CI run that went smoothly with traces: https://github.com/pharo-spec/NewTools/actions/runs/7611658534/job/20727565575 Also another example of CI run that went wrong and in which I don't know what the hell happened because there are completely different errors with identical (but less numerous) traces: https://github.com/pharo-spec/NewTools/actions/runs/7611490325/job/20727060848 |
An ugly solution that could at least fix the CI is to add an |
Let us fix it in a ugly way because there is no Morphic expert and we are working for killing morphic :) |
the problem seems to be a race condition, not directly a morphic problem, heh. |
you have |
btw, you know you can run smalltalkci locally, isn't? |
How do you do that? |
you checkout https://github.com/hpi-swa/smalltalkCI and from terminal execute something like: ./bin/smalltalkci -s Pharo64-12 /home/esteban/dev/repo/pharo-vcs/iceberg/.smalltalk.ston |
I think I fixed this error, but I now have a different (but similar) one (that was already there: https://github.com/pharo-spec/NewTools/actions/runs/7603649968/job/20705636397?pr=660) |
Well, when I do that, I get an error because the CI image cannot find Sindarin ... |
Everything should be good once pharo-spec/Spec#1511 and pharo-project/pharo#16016 will be merged |
Trying to fix #664